Skip to content

Conversation

@villfa
Copy link
Contributor

@villfa villfa commented Dec 4, 2020

No description provided.

@smoench
Copy link
Contributor

smoench commented Dec 4, 2020

I had some issue with infection after I had to upgrade it to make it runnable with PHP 8 in #508. That's the reason why I have limited PHPUnit to version 9.2.*

@villfa
Copy link
Contributor Author

villfa commented Dec 4, 2020

I'm not sure to understand. All tests pass, so what is the issue you refer to?

@villfa
Copy link
Contributor Author

villfa commented Dec 4, 2020

Also, on a side note, @theofidry could you update the settings of the project to remove the build with PHP 7.2 from the list of the required checks?

@smoench smoench mentioned this pull request Dec 7, 2020
@smoench
Copy link
Contributor

smoench commented Dec 7, 2020

I'm not sure to understand. All tests pass, so what is the issue you refer to?

Have a look at #510 CI failures

@smoench smoench mentioned this pull request Dec 7, 2020
@theofidry
Copy link
Member

Just a quick heads up as I see several notifications: last week was very busy so sorry if I was holding up anything; I'll hope to catch up a bit tonight

@theofidry theofidry merged commit ad20eeb into box-project:master Dec 7, 2020
@theofidry
Copy link
Member

Thank you @villfa

@villfa villfa deleted the upgrade-phpunit branch December 7, 2020 19:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants